Skip to content

build(deps): bump mcp lock to 1.28.1 to clear image-scan findings - #33803

Merged
yuneng-berri merged 3 commits into
litellm_internal_stagingfrom
litellm_/mcp-version-bumps-669776
Jul 18, 2026
Merged

build(deps): bump mcp lock to 1.28.1 to clear image-scan findings#33803
yuneng-berri merged 3 commits into
litellm_internal_stagingfrom
litellm_/mcp-version-bumps-669776

Conversation

@yuneng-berri

@yuneng-berri yuneng-berri commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Relevant issues

Linear ticket

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have added meaningful tests
  • My PR passes all CI/CD checks (e.g., lint, format, unit tests)
  • My PR's scope is as isolated as possible; it only solves 1 specific problem
  • I have received a Greptile Confidence Score of at least 4/5 before requesting a maintainer review (Greptile reviews automatically once the PR is opened; only comment @greptileai to re-request a review after pushing changes)

Delays in PR merge?

If you're seeing a delay in your PR being merged, ping the LiteLLM Team on Slack (#pr-review).

Screenshots / Proof of Fix

Before: the image-scan check currently fails on every PR because the shipped image carries mcp 1.26.0, e.g. this run on #33801

NAME  INSTALLED  FIXED IN  TYPE    VULNERABILITY        SEVERITY  EPSS         RISK
mcp   1.26.0     1.27.2    python  GHSA-jpw9-pfvf-9f58  High      0.3% (16th)  0.2
mcp   1.26.0     1.27.2    python  GHSA-hvrp-rf83-w775  High      0.2% (13th)  0.2
mcp   1.26.0     1.28.1    python  GHSA-vj7q-gjh5-988w  High      0.2% (7th)   0.1
[0038] ERROR discovered vulnerabilities at or above the severity threshold

After, at 52834fc (this PR's lock), building the same image and scanning it with the same grype version and flags the workflow pins (v0.114.0, --only-fixed --fail-on high):

$ docker build -f docker/Dockerfile.non_root -t litellm-image-scan:head .
$ grype litellm-image-scan:head --only-fixed --fail-on high --output table
No vulnerabilities found

The local build and scan ran on arm64; the image-scan check on this PR is the matching amd64 run. The follow-up commit a920ad5 only raises the pyproject floor and re-records it in the lock; the resolved package set, and therefore the image content, is identical to the scanned 52834fc

Type

🚄 Infrastructure

Changes

uv lock --upgrade-package mcp, taking mcp from 1.26.0 to 1.28.1 (the current latest), plus raising the pyproject.toml floor from mcp>=1.26.0,<2.0 to mcp>=1.28.1,<2.0. The floor raise is what makes the bump durable: with the old floor, any future unrelated uv lock --upgrade-package X whose dependency tree conflicts with mcp 1.28.x could legally resolve mcp back down to 1.26.0, and the regression would only surface as a red image-scan check after the fact. With the floor raised the resolver refuses outright, and installs that consume litellm[proxy] from PyPI without our lock get the same guarantee. The Docker images install with uv sync --frozen, so the shipped image picks the version up from the lock. The exclude-newer timestamp churn in the diff is uv refreshing its relative "3 days" snapshot setting, the same churn present in every recent lock update

Between 1.26.0 and 1.28.1 the upstream changes are additive (OAuth resource validation on the client, streamable HTTP stream buffering, stdio robustness fixes) plus two DeprecationWarnings introduced in 1.28.0 for the websocket transport and the experimental tasks API; litellm imports neither deprecated surface, verified by grepping every mcp import in litellm/ and enterprise/

No new tests: this bump is scanner-driven rather than feature-driven, the enforcement that the image stays clean is the image-scan workflow itself, and a static version-floor test would duplicate that gate while going stale on the next advisory. For verification, the mcp-related unit suites (about 2600 tests across the tests/test_litellm mcp directories) were run on 1.28.1, and the five files under tests/test_litellm/proxy/_experimental/mcp_server were additionally run A/B on 1.26.0 and 1.28.1 in the same environment: results are identical on both versions (337 passing, and the same 68 tests failing locally on both for environment reasons that reproduce at the staging tip without this change), so the bump changes no test outcome

Final Attestation

  • The tests check the right things, including the edge cases, and regressions in the respective real-world customer use-cases are not possible after this PR

@yuneng-berri
yuneng-berri requested a review from a team July 18, 2026 01:58
@greptile-apps

greptile-apps Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

No reviewable files after applying ignore patterns.

@codecov

codecov Bot commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@codspeed-hq

codspeed-hq Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 31 untouched benchmarks


Comparing litellm_/mcp-version-bumps-669776 (a920ad5) with litellm_internal_staging (f3d2015)1

Open in CodSpeed

Footnotes

  1. No successful run was found on litellm_internal_staging (6a26a3a) during the generation of this report, so f3d2015 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@yuneng-berri
yuneng-berri enabled auto-merge (squash) July 18, 2026 02:27
@yuneng-berri
yuneng-berri merged commit 40e914c into litellm_internal_staging Jul 18, 2026
82 of 83 checks passed
@yuneng-berri
yuneng-berri deleted the litellm_/mcp-version-bumps-669776 branch July 18, 2026 02:28
yuneng-berri added a commit that referenced this pull request Jul 18, 2026
…1.93.0 stable cut (#33847)

* fix(ci): bump pillow to 12.3.0 to resolve osv-scan CVEs (#33093)

(cherry picked from commit 20e646c)

* chore(deps): pin httplib2 and setuptools transitive floors (#33233)

Raise the constraint floors for two transitive dependencies so resolution moves them to their latest maintenance releases: httplib2 0.31.2 -> 0.32.0 and setuptools 82.0.1 -> 83.0.0. Both are pulled in only by optional integrations (Google API client, grpc tooling, lunary observability, the nvidia-riva extra), all lower-bound only, so the floors stay inside every requirer's allowed range and a default install is unaffected

(cherry picked from commit 8b32320)

* fix(anthropic/passthrough): drop incompatible temperature when downgrading adaptive thinking for pre-4.6 models (#33244)

* fix(anthropic/passthrough): drop temperature and cap thinking budget when downgrading adaptive thinking for pre-4.6 models

* test(anthropic/passthrough): use sufficient max_tokens for reasoning_effort thinking mapping

* fix(anthropic/passthrough): drop incompatible temperature when downgrading adaptive thinking for pre-4.6 models

Narrow the fix to the temperature reconciliation; the reasoning_effort
budget cap is reverted because the live translation grid relies on
budget_tokens >= max_tokens to reject unsupported effort tiers
(xhigh/max) on budget-mode models, so capping turned those 400s into
200s.

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
(cherry picked from commit 71dffc1)

* build: raise requires-python cap to <3.15 so Python 3.14 installs current releases (#33438)

* build: drop requires-python upper cap so Python 3.14 resolves to current releases

The <3.14 cap made pip on Python 3.14 fall back to litellm 1.83.7, a
pre-April release whose old auth flow fails with 400s. The cap was added
in d9a4602 because deps lacked 3.14 wheels and uv could not resolve
the 3.14 split; both are fixed now via the existing python_version
markers plus a ddtrace version split (2.x has no cp314 wheels, 3.16+
does). Verified on 3.14.5: uv sync --all-extras installs, litellm and
proxy_server import (rust bridge falls back to pure python), real
provider calls succeed sync/async/streaming, and the core-utils test
suite passes.

* build: cap requires-python at <3.15 and keep ddtrace on one major per python band

Reviewer preference to bound the supported window at the newest tested
minor rather than leaving it open-ended, and Greptile flagged the
ddtrace 3.14+ range spanning two majors; every ddtrace 4.x ships cp314
wheels so the band is now >=4.0,<5.0, matching the single-major
convention of the 2.x band.

(cherry picked from commit c6d49a8)

* build(deps): update ddtrace to the 4.x line

A single ddtrace constraint now covers every supported Python version, so this collapses the version split introduced in #33438. Also aligns the build_from_pip image pin and updates the type-only Tracer import to its current module path

(cherry picked from commit edc38ea)

* fix(docker): restore litellm-proxy-extras source dir in runtime images (#33592)

* fix(docker): restore litellm-proxy-extras source dir in runtime images

#30243 narrowed the runtime stage to an allowlist COPY, which dropped
/app/litellm-proxy-extras from the published images. Downstream
migration jobs point prisma migrate deploy at that path; with the
schema gone (or a schema with no adjacent migrations dir, where prisma
exits 0 without applying anything) those jobs went green while never
migrating the database. Restore the folder in all three runtime stages
and assert in image-scan that the schema and a non-empty migrations dir
ship at the source path

* chore(ci): drop image-scan migration-assets assertion

(cherry picked from commit 111d447)

* fix(model_armor): restore reference attachments via skip_unscannable_attachments and remove the attachment count cap (#33554)

* fix(model_armor): add skip_unscannable_attachments to allow reference-only attachments through

* fix(model_armor): wire skip_unscannable_attachments through guardrail config

* fix(model_armor): make max_file_attachments configurable and scan overflow instead of dropping

* fix(model_armor): remove the per-request attachment count cap and scan all attachments

---------

Co-authored-by: yucheng <yucheng@berri.ai>
(cherry picked from commit 0d7b0f7)

* build(rust): raise pyo3 to 0.29 so the native bridge compiles on Python 3.14 (#33798)

pyo3 0.23.5 hard-caps the interpreter at Python 3.13, so building the
native bridge against a 3.14 interpreter aborts inside pyo3-ffi's build
script before anything links. This raises pyo3 and pyo3-async-runtimes
to 0.29 (currently the newest line, and the range starting at 0.26 that
supports 3.14) and migrates the three call sites whose APIs were renamed
across that range: Python::with_gil is now Python::attach and
Python::allow_threads is now Python::detach. On a GIL-enabled interpreter
those are pure renames with identical semantics, so behavior on 3.10
through 3.13 is unchanged

Verified by compiling the native module for cp313 and cp314 and driving
it directly on both interpreters: gil_stats reports exactly one GIL
release per sync OCR call and the async path completes, matching the
0.23.5 baseline. cargo fmt, clippy, and the workspace tests pass on both
3.13 and 3.14 with the lockfile locked, and the lock churn is confined to
the pyo3 crates

Part of #26343; addresses the pyo3 build failure reported in #33116

(cherry picked from commit f3d2015)

* build(deps): allow redisvl, pypdf, and openapi-core on Python 3.14 (#33801)

Remove the python_version < '3.14' environment markers from redisvl,
pypdf, and openapi-core now that all three install and import cleanly
on 3.14. The relock is marker-only: no package version changed for any
Python branch, and the locked versions (redisvl 0.4.1, pypdf 6.13.3,
openapi-core 0.22.0) now serve 3.14 as well. semantic-router and
aurelio-sdk stay gated because every published release caps
python_requires below 3.14

(cherry picked from commit 967d934)

* build(deps): bump mcp lock to 1.28.1 to clear image-scan findings (#33803)

* build(deps): bump mcp lock to 1.28.1 to clear image-scan findings

* build(deps): require mcp>=1.28.1

(cherry picked from commit 40e914c)

* fix(proxy): source /v1/models token limits from the cost map instead of Router.get_model_group_info (#33721)

* fix(proxy): source /v1/models token limits from cost map instead of Router.get_model_group_info

Resolves the per-model get_model_group_info fan-out on GET /v1/models
(and /models) that pegged the event loop on wildcard listings (#33636).
create_model_info_response now reads max_input_tokens/max_output_tokens
from litellm.get_model_info (the static cost map) rather than the router,
which aggregated and deepcopied every deployment in a group per listed
model.

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

* test(proxy): inject model-info lookup into create_model_info_response for deterministic coverage

Inject the cost-map lookup (defaulting to litellm.get_model_info) so the
except and max_output_tokens branches are exercised deterministically and
the token-limit tests no longer hardcode mutable cost-map values.

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

* feat(proxy): surface custom deployment token limits on /v1/models via cheap index lookup

Add Router.get_configured_token_limits, an O(1) model-name index lookup that
reads a concrete deployment's configured max_input_tokens/max_output_tokens
without triggering pattern matching or deep copies. create_model_info_response
layers this over the cost map so custom deployments absent from the cost map
still surface their limits, and admin-configured limits override cost-map
defaults, while wildcard-expanded names stay on the fast path.

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

---------

Co-authored-by: ryan <ryan@berri.ai>
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
(cherry picked from commit 8536e3b)

---------

Co-authored-by: yucheng-berri <yucheng@berri.ai>
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: ryan-crabbe-berri <ryan@berri.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants